sync with master#420
Merged
albertwang-ibm merged 3 commits intoAlbert-Master-Newfrom Nov 15, 2019
Merged
Conversation
* Issue #329 - turn down logging Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #355 - Fixed semantic errors in FHIRPathType.isAssignableFrom Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #355 - Added unit tests, fixed tabs/spaces issue. Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #355 - Fixed indentation Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #355 - updated per PR feedback Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #389 - upgrade FHIRPath grammar version Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #389 - refactor quantity processing Signed-off-by: John T.E. Timm <johntimm@us.ibm.com> * Issue #389 - Updates per PR feedback Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>
1. Updated the SearchPerformanceTest so that the observations have components with values...just to easily create many observations which helps verify the search behavior. This change could be removed if we want. 2. Deleted unused variants of searchForIds and searchByIds. I think these came from the original "basic" schema that was removed for #93 but they were no longer in use and should be safe to delete. 3. Introduced ORDER BY CASE clause to ResourceDAOImpl.searchByIds. I found that DERBY is always returning the results in the same order as the resource ids provided in the "IN" clause, but in general we shouldn't rely on that. I tested the CASE statements on derby with up to 1000 results, but we need to run a similar test on Db2 to ensure it works there as well. Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
issue #418 - add order by clause to searchByIds to ensure sorted results
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.